Skip to content

Comments

PHP 8 Support#106

Open
utopiabound wants to merge 11 commits intopellcorp:masterfrom
utopiabound:php8
Open

PHP 8 Support#106
utopiabound wants to merge 11 commits intopellcorp:masterfrom
utopiabound:php8

Conversation

@utopiabound
Copy link
Contributor

Support for PHP4 and PHP5 is dropped
Convert while (list($key, $var) = each($array)) to foreach()
Convert any array {} lookups to []
Upgrade libchart to 1.3 (final)
Upgrade Snoopy to 2.0.0 (final)
Upgrade PHPMailer to 5.2.28 (final before 6.x rewrite)
Upgrade PHPThumb to 1.7.17 (
Remove earlier libchart versions.
Fix many uninitialized values (fixes warnings in logs)

Support for PHP4 and PHP5 is dropped
Convert while (list($key, $var) = each($array)) to foreach()
Convert any array {} lookups to []
Upgrade libchart to 1.3 (final)
Upgrade Snoopy to 2.0.0 (final)
Upgrade PHPMailer to 5.2.28 (final before 6.x rewrite)
Upgrade PHPThumb to 1.7.17 (
Remove earlier libchart versions.
Fix many uninitialized values (fixes warnings in logs)

Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Copy link

@NicFord1 NicFord1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Came across two code issues.

phplint: "php -l" on each php file
A few more warnings, and bugs squashed.

Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Remove some unused "$key" variables in foreach statements.
Use ?? operator instead of checking isset() in some places.

Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
borrow.php: Fix incorrect constant name

Fix warnings of uninitialized variables.

Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Also more warnings

Signed-off-by: Nathaniel Clark <Nathaniel.Clark@misrule.us>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants